-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat [#19] 나의 챌린지 뷰 UI 구현 #25
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p3. 벌써 뷰가 만들어진 것 같습니다.. 역시! 수고하셨습니다😎
HMH_iOS/HMH_iOS/Presentation/Challenge/Cells/AppListCollectionViewCell.swift
Outdated
Show resolved
Hide resolved
HMH_iOS/HMH_iOS/Presentation/Challenge/Cells/DateCollectionViewCell.swift
Outdated
Show resolved
Hide resolved
HMH_iOS/HMH_iOS/Presentation/Challenge/ViewControllers/ChallengeViewController.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
너무 길어요,, 너무 고생하셧어요 ㅠㅠㅠ 제발 자세요 ㅠㅠㅠ 👍
HMH_iOS/HMH_iOS/Presentation/Challenge/Cells/AppListCollectionViewCell.swift
Outdated
Show resolved
Hide resolved
HMH_iOS/HMH_iOS/Presentation/Challenge/Cells/AppListCollectionViewCell.swift
Outdated
Show resolved
Hide resolved
HMH_iOS/HMH_iOS/Presentation/Challenge/Cells/AppListCollectionViewCell.swift
Outdated
Show resolved
Hide resolved
static let identifer = "DateCollectionViewCell" | ||
|
||
let dateLabel = UILabel().then { | ||
$0.text = "1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p5. 까먹지 않기!
👾 작업 내용
"나의 챌린지 뷰" UI를 구현했습니다.
🚀 PR Point
Compositional Layout
을 사용하여 구현했습니다.NsCollectionViewLayoutSection
을 만들 때,adjusted
를 사용하면 오히려 레이아웃이 깨지는 현상을 발견하여 , 그 부분에 한하여 사용하지 않았습니다.Model
을 만들었습니다Height
을 지정해주는 것이 추후 재사용 시 편리할 것 같습니다. 수정 부탁드립니다. @kim-seonwoo📸 스크린샷
✅ Issue
Resolved #19